Skip to content

node_exporter: use dehydrated when use_https is enabled#428

Merged
aagbsn merged 1 commit into
mainfrom
fix_missing_dehydrated
May 18, 2026
Merged

node_exporter: use dehydrated when use_https is enabled#428
aagbsn merged 1 commit into
mainfrom
fix_missing_dehydrated

Conversation

@aagbsn
Copy link
Copy Markdown
Contributor

@aagbsn aagbsn commented May 15, 2026

some of our playbooks also specify the dehydrated role - but if it isn't, then no certificate is created and this role would then fail.

@github-actions
Copy link
Copy Markdown

github-actions Bot commented May 15, 2026

Ansible Run Output 🤖

Ansible Playbook Recap 🔍

Ansible playbook output 📖success

Show Execution 

$ ansible-playbook playbook.yml --check --diff -i ../tf/modules/ansible_inventory/inventories/inventory-dev.ini
[WARNING]: provided hosts list is empty, only localhost is available. Note that the implicit localhost does not match 'all'
[ERROR]: the role 'geerlingguy.docker' was not found in /home/runner/work/devops/devops/ansible/roles:/home/runner/.ansible/roles:/usr/share/ansible/roles:/etc/ansible/roles:/home/runner/work/devops/devops/ansible
Origin: /home/runner/work/devops/devops/ansible/deploy-testlists.yml:16:7

14         node_exporter_host: "0.0.0.0"
15         node_exporter_options: ""
16     - role: geerlingguy.docker
         ^ column 7
Pusher @aagbsn
Action pull_request
Working Directory
Workflow .github/workflows/check_ansible.yml
Last updated Fri, 15 May 2026 09:42:37 GMT

LDiazN
LDiazN approved these changes May 18, 2026
View reviewed changes
Copy link
Copy Markdown
Contributor

@LDiazN LDiazN left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@aagbsn aagbsn merged commit 4d13c31 into main May 18, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@LDiazN LDiazN LDiazN approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@aagbsn @LDiazN